Skip to content

Conversation

kateinoigakukun
Copy link
Member

Those files are recently added to the toolchain, and they are required to build Swift code that uses Foundation on WebAssembly.

We had been manually specifying the paths to copy from the target package into the SDK, but we now build host-agnostic SDKS. So we can just copy the whole usr/lib from the target Swift package into the SDK bundle instead of adding individual new paths. The host-specific build path is still used for 5.10 SDKs, so keep the old code as is for now, but it will be removed in the future.

…K bundle

Those files are recently added to the toolchain, and they are required
to build Swift code that uses Foundation on WebAssembly.

We had been manually specifying the paths to copy from the target
package into the SDK, but we now build host-agnostic SDKS.
So we can just copy the whole `usr/lib` from the target Swift package
into the SDK bundle instead of adding individual new paths.
The host-specific build path is still used for 5.10 SDKs, so keep the
old code as is for now, but it will be removed in the future.
@kateinoigakukun
Copy link
Member Author

@swift-ci test

@MaxDesiatov
Copy link
Contributor

@swift-ci test

@MaxDesiatov
Copy link
Contributor

@swift-ci test macos

@MaxDesiatov
Copy link
Contributor

@swift-ci test

@MaxDesiatov MaxDesiatov added the dependencies Changes to dependencies label Aug 28, 2024
@kateinoigakukun kateinoigakukun marked this pull request as ready for review August 28, 2024 17:26
@kateinoigakukun
Copy link
Member Author

Verified in swiftwasm/swiftwasm-build#418

@MaxDesiatov
Copy link
Contributor

@swift-ci test

@MaxDesiatov MaxDesiatov enabled auto-merge (squash) August 28, 2024 17:28
@MaxDesiatov MaxDesiatov disabled auto-merge August 28, 2024 17:28
@kateinoigakukun kateinoigakukun merged commit 13c0c51 into swiftlang:main Aug 28, 2024
3 checks passed
@kateinoigakukun kateinoigakukun deleted the yt/swift-foundation branch August 30, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Changes to dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants